A11y: fix typo when initializing a container cell accessible

Gtk_container_cell_widget_set should chain up to its parent's set
function, not its parent's unset function. This was resulting in
accessibles being erroneously marked defunct after being created.

Updated Properties Hebrew translation

Updated Hebrew translation

x11: Handle obsolete selection requests without crashing

The ICCCM says:

If the specified property is None, the requestor is an obsolete client.
Owners are encouraged to support these clients by using the specified
target atom as the property name to be used for the reply.

Lets do that, instead of crashing.

https://bugzilla.gnome.org/show_bug.cgi?id=740613

The previous fix for this issue in 732af31424b8f382d was incomplete.

Trivial doc fixes

Remove two erroneous quotation marks.

GtkFileChooserWidget: Fix a crash for when the widget was unrefed right after setting a current folder.

Keep a reference to the widget while the async operation is ongoing.

https://bugzilla.gnome.org/show_bug.cgi?id=725164

GtkFileChooserWidget: remove unused fields and cleanup code.

These fields are never set, so we don't need the code to clean them up.

https://bugzilla.gnome.org/show_bug.cgi?id=725164

wayland: Don't claim to support selection notification

We don't support selection notification, so don't claim
we do, only to fail later.

https://bugzilla.gnome.org/show_bug.cgi?id=743229

Initialize variables

No need to risk valgrind complaints about initialized
values.

https://bugzilla.gnome.org/show_bug.cgi?id=743422

GtkTreeModel: Documentation fix

https://bugzilla.gnome.org/show_bug.cgi?id=743286

Fix transfer annotation for tree_model parameter of gtk_tree_get_row_drag_data() and add nullable, optional

The GtkSelectionData does not own the model set with
gtk_tree_set_row_drag_data so change the out param to "transfer none".
This fixes a crash after multiple DnD actions due to the bindings
stealing a reference each time this function is called.

This also adds nullable and optional annotations for tree_model and path while at it.

https://bugzilla.gnome.org/show_bug.cgi?id=743193

gtkdnd: Set the dnd window's screen before its visual

This makes dnd icons work for widgets on a !default screen.

https://bugzilla.gnome.org/show_bug.cgi?id=743393

file chooser: search improvements

Make it so that typing switches to search unless we have a better
use for the key events, and stop search if the entry is emptied.

file chooser: Turn off typeahead search

It is the third kind of navigation entry in the file chooser after
the location entry and search, and it interacts badly with both
of the others.

gtk-demo: Show search entry signals

Use the new API in GtkSearchBar if we can

Sadly, GtkSearchBar allows connecting plain entries, so we have
to keep the old code around.

inspector: Use some of the new GtkSearchEntry API

Avoid repetitive, manual event handling in various places
in GtkInspector.

GtkSearchEntry: Add more API

Add ::next-match, ::previous-match and ::stop-search keybinding
signals that are bound to Ctrl-g, Ctrl-Shift-g and Escape. Also
add a gtk_search_entry_handle_event() function to handle key events.

Updated Icelandic translation

inspector: Make gl extension rows unactivatable

Updated POTFILES.skip

roundedbox: Treat path segments differently

If a side of the box is 0px wide, make the corners owned by the adjacent
sides. This avoids spilling over of unwanted colors from the 0-width
side into the corner.

New test for this case is included.

widget: Compute clip of children properly

For widgets with a window, we need to adjust the allocation of their
children.

widget: Correctly handle which widgets to include in the clip

Previously, we would not include any child widget on the first
allocation, which happens right after realize(), but before map(). No
widget is drawable at that point.

widget: Add debug spew for setting the clip

headerbar: Implement clipping

border: Don't special case according to border styles

This is now done (as it should) in the compute code.

css: Move "border-style: none" special case to CssValueNumber

This fixes the background being shifted by the border-width even though
the border-style was none.

stylecontext: Don't do unnecessary workarounds

when calling get_padding/border/margin, there is no need to go through
the fallback API.

This is a huge speedup for certain benchmarks.

render: fix arrow orientation

The rotation code in the draw_arrow function was assuming that the arrow
would be drawn pointing upwards but it was pointing to the right
resulting in the rotated arrows pointing in the wrong direction.

The recent refactoring caused a pi/2 rotation to be lost. Rather than
adding that back somehwere (to lose it again in the future), we just
draw the arrow pointing upwards as it is expected to do with a 0 angle.

3.15.4

More updates

image: Compute clip properly

Use the helper function we have for this, don't do it manually. This has
the side effect of also supporting icon transforms.

render: Implement -gtk-icon-transform for icon surfaces

Yes, I like playing around. To enjoy, add this CSS to your application
of choice (preferrably glade or something with lots of images):

GtkImage { animation: spin 2s linear infinite; }

You can thank me later.

render: Split icon surface rendering out

Just like normal icons, icon surfaces belong into gtkrendericon.c.

render: Split icon rendering into its own file

render: Remove spinner special-cases

The spinner is a regular builtin image now. There is no need to go
through the shadows code manually anymore as regular items do get
shadows automatically.

This also allows simplifying the actual spinner drawing code so that it
actually works.

render: Render builtin images as regular CSS images

This not only reduces code, but also allows for builtin images to have
shadows or be transformed using -gtk-icon-transform.

render: Don't require passing CSS properties to builtin images

Instead, make the buiultin image get those properties itself in the
compute vfunc.

css: Move icon properties to the end

The order in which properties are defined depends on the order in which
they are computed. And that means that properties can only depend on
other properties that are defined before them.

The next patches will need this reordering.

render: Only render 4 different arrows

Instead of supporting every angle, just support top, right, bottom and
left and round the angle to one of those directions.

Adwaita overrides arrows anyway and doesn't even look at the angle, so
this should not be a problem.

render: Don't pass junction sides to builtin image

Instead, split the grip into 8 different builtin images (one for each
side and corner).

render: Extend builtin images to account for states

We add to the enum to account for the images generated for different
states.

render: Move code for rendering builtin images

Also, make it have a generic entry point with
gtk_css_image_builtin_draw().

The only feature lost so is the drawing of shadows for spinners, but
that will come back later.

inspector: Remove unused hash table

inspector: Remove unused treemodel properties

As a side effect, we can use _gtk_css_section_to_string() to format the
section and save a lot of code.

inspector: Use GtkCssStyle and gtk_css_value_to_string()

render: Move border rendering into its own file

gtkrenderborder.c now contains the implementation for
  gtk_css_style_render_border
  gtk_css_style_render_outline

render: Remove unused header

render: Rename gtkborderimage.[ch]

I want the gtkrender prefix for all filenames about rendering.

render: Use a GtkCssStyle instead of GtkStyleContext for focus

render: Rename gtkthemingbackground.[ch]

I want the gtkrender prefix for all filenames about rendering.

render: Take a GtkCssStyle for rendering frames

... and move the whole function into gtkthemingbackground.c

borderimage: Use a GtkCssStyle instead of GtkStyleContext

themingbackground: Use a GtkCssStyle instead of GtkStyleContext

roundedbox: Take a GtkCssStyle instead of a GtkStyleContext

This is in preparation for iusing GtkCssStyle instead of GtkStyleContext
in render functions.

stylecontext: Rename function and export it.

The name now better reflects what it does, since GtkCssComputedValues
was renamed to GtkCssStyle, too.

reftests: Fix border-style-none reftest

Unset all CSS; Adwaita was messing up things.

dnd: Draw dnd icon with regular code

Overriding the window background is ugly because the window background
is owned by GTK and can be reset at any time.

https://bugzilla.gnome.org/show_bug.cgi?id=742646

stylecontext: Keep track of the CSS ID

This is necessary since we do the new caching and need to distinguish
between styles with different IDs.

Fixes various test cases.

Updated Lithuanian translation

Updated Spanish translation

Updated Spanish translation

path bar: Don't segfault if we get disposed quickly

The file chooser code now calls into the path bar in init,
starting an async operation. This unveiled that the path bar
code was not safe against being disposed early, by causing
a crahsh of the objects-finalize test.
Fix this by making the callback check for CANCELLED.

Updates for 3.15.4

places sidebar: Drop the top margin

This looks odd in combination with the top underflow.

x11: Fetch vendor/product identifiers for input devices

These are retrieved from XInput device properties.

https://bugzilla.gnome.org/show_bug.cgi?id=740758

device: Add property/getters for vendor/product identifiers

These are a construct only properties, expected to be filled in from
platform specific code.

https://bugzilla.gnome.org/show_bug.cgi?id=740758

gtk-demo: Make editable cells demo more robust

When removing all rows, trying to add rows would not work
and throw criticals. This is fallout from a recent change
to insert rows at the right position. Fix this by handling
the 'empty model' case separately.

https://bugzilla.gnome.org/show_bug.cgi?id=743157

scrolledwindow: Don't displace invalidation by the widget allocation

This was left unmodified when GtkScrolledWindow was made a windowed widget.
Displacing here by the widget allocation is not necessary anymore, since
we are invalidating the window set at these coordinates.

This patch is a simplification of a previous one by Timm Bäder.

https://bugzilla.gnome.org/show_bug.cgi?id=742243

stylecontext: Fix FALSE/TRUE mixup

Introduced in e649c8773f0357df19f4e28aabf40a203a39c409. It's obviously
the root node right there.

scrolled window: Reduce the hover proximity distance further

file chooser: Fix up some focus issues

Better to switch the visible stack child early, since GtkStack
has its own focus handling now.

gtkimagemenuitem: create widget for action icons lazily

GtkImageMenuItem always packs a GtkImage when it is synced with a
related action. This is unnecessary when the action doesn't have an icon
set and results in excess padding when a GtkImageMenuItem is put in a
menu bar (GtkUIManager does this).

Fix this by not creating the GtkImage for icon-less actions.

https://bugzilla.gnome.org/show_bug.cgi?id=742181

render: Replace gtk_style_context_get_border/padding()

Use _gtk_style_context_peek_property() instead.

render: Get rid of gtk_style_context_get_color()

Use _gtk_style_context_peek_property() instead.

Adwaita: Make :active button gradient have 3 stops

Just add a 3rd stop in the place where the gradients for the other
states have a 3rd stop. When running transitions between states, this
avoids cross-fades and instead uses the much faster gradient
transitioning code.

stylecontext: Be more graceful when save() isn't matched with restore()

Instead of asserting, just print a g_warning() and try to work around
the problem.
I hope that g_warning() isn't too spammy for people that are hit with
it.

Also clarify the docs that not restore()ing after a save() is a bad
idea.

https://bugzilla.gnome.org/show_bug.cgi?id=743101

stylecontext: Explicitly specify root node

When creating the query path, explicitly specify if it's for a root node
or for a child. Relying on gtk_style_context_is_saved() is unreliable
(for example when updating the cache).

file chooser: Make search work again

The last round of filechooser redesign work had left
search somewhat dysfunctional. This commit fixes things
up enough that search is once again a working feature
of the file chooser.

file chooser: Drop the location label

Bringing us close to the new design.

file chooser: Don't show 'Enter Location' in save modes

In these modes, the entry is permanently visible anyway.

file chooser: Make Ctrl-L work in recent

Just switch to browse mode when we see Ctrl-L in recent mode.
https://bugzilla.gnome.org/show_bug.cgi?id=660450

file chooser: Ensure the path bar is initialized

There are some cases where we can end up showing the path
bar without ever setting a file on it. One example is:
1) Open file chooser
2) Hit ~
3) Hit Ctrl-L
While this is a bit of a workaround, at least it makes sure
that we never end up with an empty space where the path bar
is supposed to be.

trivial refactoring

file chooser: Make ~ and / work in recent

Complained about in
https://bugzilla.gnome.org/show_bug.cgi?id=743117

Fix a typo in the docs

https://bugzilla.gnome.org/show_bug.cgi?id=743037

broadway: Only use abstract sockets when they are supported

GIO has a function to check whether abstract sockets are
supported, so use it. This should make broadway work on
Freebsd again.

https://bugzilla.gnome.org/show_bug.cgi?id=742980

Make Broadway work on freebsd

It seems that posix_fallocate gives an ENODEV error when
called on an fd opened with shm_open on freebsd. Fix up
the error check to only trigger if we get ENOSPC.

https://bugzilla.gnome.org/show_bug.cgi?id=742980

Fine-tune the proximity hovering for scrollbars

Reduce the distance at which we expand the scrollbar,
and add some hysteresis.

Updated Greek translation

Remove some outdated references

We shouldn't recommend gtk_widget_modify_bg() or
gtk_style_set_background() anymore.

Adjust gtk_status_icon_position_menu() annotation to match GtkMenuPositionFunc again

https://bugzilla.gnome.org/show_bug.cgi?id=674537

https://bugzilla.gnome.org/show_bug.cgi?id=742940

Fix event->button.y_root when using mouse buttons and dnd canceled

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742785

Fix save_hot_y init in gtk_drag_get_icon()

Signed-off-by: Maks Naumov <maksqwe1@ukr.net>
https://bugzilla.gnome.org/show_bug.cgi?id=742786

Updated Basque language

Prefer the Wayland backend

This does not affect X11 sessions, but has the advantage that
we get much more testing of the Wayland backend in Wayland
sessions.

inspector: Remove a bunch of unused variables

stylecontext: Remove a bunch of return_if_fail()s

They're gonna be in the way when refactoring and we can live without
them.

stylecontext: Do not transition from the default style

This gets rid of a bunch of awkward transitions. It's not a perfect
solution to the problem of "should we transition from this state" but it
gets rid of the ugliest offenders.